Skip to content

Conversation

CDRussell
Copy link
Member

@CDRussell CDRussell commented Sep 19, 2025

Task/Issue URL: https://app.asana.com/1/137249556945/project/608920331025315/task/1211197010246723?focus=true

Description

Adds ability to parse a Google Takeout zip file containing exported bookmarks. This PR only adds underlying logic and dev settings to test it; prod UI isn't changed.

Steps to test this PR

Importing a valid file

  • Generate a Takeout zip file containing bookmarks. See "Instructions for generating a Takeout zip file" below.
  • Fresh install of internal build type
  • launch Autofill Dev Settings, and scroll to Import Bookmarks dev settings 🆕
  • tap Choose zip file (downloaded from Takeout) and choose the Takeout zip; verify the bookmarks imported using the View Bookmarks button

Importing an invalid file

  • tap Choose zip file (downloaded from Takeout) and choose a random other file (either not a Takeout zip or one that doesn't contain bookmarks) and verify it just reports it as invalid as a snackbar

Instructions for generating a Takeout zip file

Tap Launch Google Takeout (normal tab) from Autofill Dev Settings and use this to generate a zip file for testing:
- [ ] sign in
- [ ] Tap on Deselect All
- [ ] Scroll down to Chrome, and check it
- [ ] Optionally, Tap on All Chrome data included and choose only bookmarks
- [ ] Scroll all the way down to the bottom and tap Next step
- [ ] Leave all settings as is, and choose Create export
- [ ] Wait a few seconds, then refresh the page until you see Your latest export shows the export you just created (at the top of the page)
- [ ] Tap Download and confirm you want to save it

@CDRussell CDRussell force-pushed the feature/craig/streamline_import_bookmarks_takeout_zip branch 5 times, most recently from 29e253d to 1b6e8bb Compare September 19, 2025 12:22
@CDRussell CDRussell marked this pull request as ready for review September 19, 2025 12:40
@CDRussell CDRussell changed the title Add ability to parse a Takeout zip file and extra bookmarks Add ability to parse a Takeout zip file and extract bookmarks Sep 19, 2025
@CDRussell CDRussell changed the title Add ability to parse a Takeout zip file and extract bookmarks Add ability to parse a Takeout zip file and extract+import bookmarks Sep 19, 2025
@CDRussell CDRussell force-pushed the feature/craig/streamline_import_add_feature_flag branch from eda2570 to 1b8cac9 Compare September 19, 2025 15:17
@CDRussell CDRussell force-pushed the feature/craig/streamline_import_bookmarks_takeout_zip branch from 1b6e8bb to 01d5515 Compare September 19, 2025 15:17
Copy link
Contributor

@cmonfortep cmonfortep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Posting review. Moving into testing. I will report after that.

Copy link
Contributor

@cmonfortep cmonfortep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good with testing 👍

Approving, but would like to validate the proposed approach around reading the extracted file, or if this is something you already validated.

Copy link
Contributor

@cmonfortep cmonfortep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

didn't approve 😅

@CDRussell CDRussell changed the base branch from feature/craig/streamline_import_add_feature_flag to graphite-base/6820 September 23, 2025 11:31
@CDRussell CDRussell force-pushed the feature/craig/streamline_import_bookmarks_takeout_zip branch from 01d5515 to 1dfcbd0 Compare September 23, 2025 11:32
@graphite-app graphite-app bot changed the base branch from graphite-base/6820 to develop September 23, 2025 11:32
@CDRussell CDRussell force-pushed the feature/craig/streamline_import_bookmarks_takeout_zip branch 2 times, most recently from 6f10a11 to 1dfcbd0 Compare September 23, 2025 11:35
@CDRussell CDRussell force-pushed the feature/craig/streamline_import_bookmarks_takeout_zip branch from 1dfcbd0 to 0322886 Compare September 23, 2025 12:01
@CDRussell CDRussell merged commit c3b2c4a into develop Sep 23, 2025
7 checks passed
@CDRussell CDRussell deleted the feature/craig/streamline_import_bookmarks_takeout_zip branch September 23, 2025 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants